From: Florian Müllner Date: Thu, 25 Jul 2019 10:24:50 +0000 (+0200) Subject: tweener: Use new adjustAnimationTime() helper X-Git-Tag: archive/raspbian/50.5-1+rpi1~1^2~839^2^2~55 X-Git-Url: https://dgit.raspbian.org/%22http://www.example.com/cgi/%22/%22http:/www.example.com/cgi/%22?a=commitdiff_plain;h=99b4e047ddc836e715214201d14a65cafdb40e72;p=gnome-shell.git tweener: Use new adjustAnimationTime() helper Now that we have a new helper function for adjusting animation times, make Tweener use it so that we keep the code in a single place. https://gitlab.gnome.org/GNOME/gnome-shell/merge_requests/22 --- diff --git a/js/ui/tweener.js b/js/ui/tweener.js index d9fab4f6a..221438c1b 100644 --- a/js/ui/tweener.js +++ b/js/ui/tweener.js @@ -3,10 +3,12 @@ pauseTweens, resumeTweens, registerSpecialProperty, registerSpecialPropertyModifier, registerSpecialPropertySplitter */ -const { Clutter, GLib, Shell, St } = imports.gi; +const { Clutter, GLib, Shell } = imports.gi; const Signals = imports.signals; const Tweener = imports.tweener.tweener; +const { adjustAnimationTime } = imports.ui.environment; + // This is a wrapper around imports.tweener.tweener that adds a bit of // Clutter integration. If the tweening target is a Clutter.Actor, then // the tweenings will automatically be removed if the actor is destroyed. @@ -50,10 +52,11 @@ function _wrapTweening(target, tweeningParameters) { } } - if (!St.Settings.get().enable_animations) { - tweeningParameters['time'] = 0.000001; - tweeningParameters['delay'] = 0.000001; - } + let { time, delay } = tweeningParameters; + if (!isNaN(time)) + tweeningParameters['time'] = adjustAnimationTime(1000 * time) / 1000; + if (!isNaN(delay)) + tweeningParameters['delay'] = adjustAnimationTime(1000 * delay) / 1000; _addHandler(target, tweeningParameters, 'onComplete', _tweenCompleted); } @@ -211,9 +214,6 @@ var ClutterFrameTicker = class { } start() { - let settings = St.Settings.get(); - if (settings.slow_down_factor > 0) - Tweener.setTimeScale(1 / settings.slow_down_factor); this._timeline.start(); global.begin_work(); }